Skip to content

Deep Review UX improvements and flow-chat terminal card grouping#528

Merged
limityan merged 4 commits intoGCWing:mainfrom
limityan:yanzhn/fix
Apr 25, 2026
Merged

Deep Review UX improvements and flow-chat terminal card grouping#528
limityan merged 4 commits intoGCWing:mainfrom
limityan:yanzhn/fix

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

Deep Review UX improvements and flow-chat terminal card grouping

Changes

Deep Review UX

  • Rename review agent funNames to professional titles (Logic/Performance/Security Reviewer, Review Arbiter) with i18n
  • Remove stop-review confirmation dialog for direct action
  • Pre-create subagent placeholder cards on model round start for better perceived parallelism
  • Fix subagent card expanded height and reduce text padding
  • Float DeepReviewActionBar as independent card with gap from bottom panel
  • Change deep review fix completion to dismiss action bar instead of showing fix_completed state
  • Update review agent prompts for third-party judge perspective
  • Fix ReviewTeamPage responsibility list key uniqueness

Review Team Page Refactor

  • Merge member detail into expandable inline cards with chevron, remove nested body frames
  • Fix policy panel overflow with inherit border-radius and auto-fit grid
  • Update test assertion to match new inline detail rendering

Flow-chat Terminal Card Grouping

  • Switch TerminalToolCard / GitToolDisplay to CompactToolCard for tighter consistent layout
  • Add TerminalGroupRenderer to merge consecutive Bash commands into collapsible group with status summary
  • Add useTerminalGroupState hook for terminal group expansion state
  • Extend ModelRoundItem grouping logic to support terminal-type groups
  • Reduce CompactToolCard margin-bottom (0.6rem -> 0.35rem)
  • Add terminalRegion i18n (en-US / zh-CN / zh-TW)

…e bash commands

- TerminalToolCard: switch from BaseToolCard to CompactToolCard for tighter layout
- GitToolDisplay: switch from BaseToolCard to CompactToolCard for consistent styling
- TerminalGroupRenderer: new component to merge consecutive Bash commands into a collapsible group with status summary
- ModelRoundItem: extend grouping logic to support terminal-type groups
- FlowChatContext + ModernFlowChatContainer: add terminal group collapse state management
- useTerminalGroupState: new hook for terminal group expansion state
- CompactToolCard.scss: reduce tool wrapper margin-bottom from 0.6rem to 0.35rem
- TerminalGroupRenderer.scss: tighter spacing for cards inside terminal groups
- i18n: add terminalRegion translations (en-US + zh-CN)
… frames

- Fix policy panel overflow by using inherit border-radius and auto-fit grid
- Remove nested ConfigPageSection body frames from overview and members sections
- Merge member detail into expandable cards with expand/collapse chevron
- Update test assertion to match new inline detail rendering
- Rename review agent funNames to professional titles (Logic/Performance/Security Reviewer, Review Arbiter) with i18n
- Remove stop-review confirmation dialog for direct action
- Pre-create subagent placeholder cards on model round start for better perceived parallelism
- Fix subagent card expanded height and reduce text padding
- Change deep review fix completion to dismiss action bar instead of showing fix_completed state
- Float DeepReviewActionBar as independent card with gap from bottom panel
- Update review agent prompts for third-party judge perspective
- Fix ReviewTeamPage responsibility list key uniqueness
@limityan limityan merged commit e931339 into GCWing:main Apr 25, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant